Skip to content

feat: add missing import link to the toolbar upper links - #430

Merged
sheabunge merged 2 commits into
core-betafrom
feat/add-import-link-to-toolbar/core
Aug 4, 2026
Merged

feat: add missing import link to the toolbar upper links#430
sheabunge merged 2 commits into
core-betafrom
feat/add-import-link-to-toolbar/core

Conversation

@ramiy

@ramiy ramiy commented Aug 3, 2026

Copy link
Copy Markdown
Contributor
image

@ramiy
ramiy requested a review from imantsk August 3, 2026 15:02
@code-snippets-bot

code-snippets-bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor
  • Low — src/js/components/common/Toolbar.tsx:46: The context-free “Import” label can use the wrong grammatical form in translations and is inconsistent with the existing import navigation strings. Use _x('Import', 'snippets', 'code-snippets') and import _x.

@ramiy

ramiy commented Aug 4, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author
  • Low — src/js/components/common/Toolbar.tsx:46: The context-free “Import” label can use the wrong grammatical form in translations and is inconsistent with the existing import navigation strings. Use _x('Import', 'snippets', 'code-snippets') and import _x.

Fixed.

@code-snippets-bot

code-snippets-bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

LGTM! Ready to merge!

@sheabunge

sheabunge commented Aug 4, 2026

Copy link
Copy Markdown
Member

Good catch!

@sheabunge
sheabunge merged commit 0a22f06 into core-beta Aug 4, 2026
9 checks passed
@sheabunge
sheabunge deleted the feat/add-import-link-to-toolbar/core branch August 4, 2026 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@imantsk imantsk Awaiting requested review from imantsk

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ramiy @code-snippets-bot @sheabunge